Skip to content

Swift: Adopt the smaller macOS host libraries - #22452

Open
henrymercer wants to merge 4 commits into
mainfrom
henrymercer/swift-resource-dir-thinning
Open

Swift: Adopt the smaller macOS host libraries#22452
henrymercer wants to merge 4 commits into
mainfrom
henrymercer/swift-resource-dir-thinning

Conversation

@henrymercer

@henrymercer henrymercer commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Replace the macOS Swift resource archive with the host-only thinned version and remove the staging override. This reduces the archive from 548 MB to 433 MB.

Only the host libraries are thinned to x64. Target SDK/runtime libraries and module metadata are unchanged, and Swift remains at 6.3.3.

Point at the staging artifact built with the x86_64-only thinning from
semmle-code.  Must be reverted before merging.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 03df685d-b0f4-4920-8c7c-8e70c6c868bf
@github-actions github-actions Bot added the Swift label Aug 27, 2026
The semmle-code change now thins only `usr/lib/swift/host`, so test
that artifact rather than the earlier one which also thinned the
per-target libraries.
@henrymercer henrymercer changed the title Swift: Test the thinned macOS resource dir Swift: Adopt the smaller macOS host libraries Sep 10, 2026
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@henrymercer
henrymercer marked this pull request as ready for review September 10, 2026 12:58
Copilot AI balanced review requested due to automatic review settings September 10, 2026 12:58
@henrymercer
henrymercer requested a review from a team as a code owner September 10, 2026 12:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🔵 Needs a closer look

The change updates a large binary/toolchain resource via Git LFS, and the correctness of the new archive’s contents (beyond the pointer metadata) can’t be validated from the diff alone.

Review tier: Lite
Findings: None

What changed in this PR

Updates the Swift macOS resource archive used by the Swift toolchain integration to a smaller, host-thinned variant, reducing the checked-in Git LFS artifact size while keeping the Swift version pinned at 6.3.3.

Changes:

  • Replaces swift/third_party/resources/resource-dir-macos.zip (Git LFS pointer) with a new object ID and smaller recorded size.
File Description
swift/​third_party/​resources/​resource-dir-macos.zip Updates the Git LFS pointer to the new, smaller macOS Swift resource archive.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants